-
Notifications
You must be signed in to change notification settings - Fork 1.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Replace non-existing manifests and prevent them from changing in the future #6425
Replace non-existing manifests and prevent them from changing in the future #6425
Conversation
…future Signed-off-by: Tatiana Krishtop <tkrishto@redhat.com>
@tkrishtop I was going to raise a follow up PR today, thanks for putting this together. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Closing and reopening to try and pick up the changes that were merged in #6422 |
/hold cancel |
hi @everettraven I merged the master branch into fix_digests to pick the change of @acornett21 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
Description of the change:
This PR is to be merged after the PR @acornett21 : #6422
Motivation for the change:
To support the integration of DCI, Preflight, and Operator-SDK in disconnected environments, we have to use the digests and not tags for scorecard-storage and scorecard-untar. We also have to protect these digests from being overwritten during the release process. All the details are here: #6417.
Checklist
If the pull request includes user-facing changes, extra documentation is required:
changelog/fragments
(seechangelog/fragments/00-template.yaml
)website/content/en/docs